Skip to content

feat(postgres): modify the docker base image as postgres:11-alpine#5

Merged
Cryptophobia merged 8 commits into
teamhephy:masterfrom
duanhongyi:master
Nov 28, 2018
Merged

feat(postgres): modify the docker base image as postgres:11-alpine#5
Cryptophobia merged 8 commits into
teamhephy:masterfrom
duanhongyi:master

Conversation

@duanhongyi

@duanhongyi duanhongyi commented Nov 17, 2018

Copy link
Copy Markdown
Member

Using postgres:11-alpine instead of the postgres:11, the size is close to the original 1/3.

@duanhongyi duanhongyi changed the title feat(postgres): modify the docker base image as postgres:11-apline feat(postgres): modify the docker base image as postgres:11-alpine Nov 17, 2018
@duanhongyi duanhongyi force-pushed the master branch 3 times, most recently from 1d1d8a0 to e1b0c3b Compare November 17, 2018 23:14
@Cryptophobia

Copy link
Copy Markdown
Member

Looks good! It's exciting that it's only 1/3 the size of the original postgres image. @duanhongyi , is this ready for review and merge?

@duanhongyi

Copy link
Copy Markdown
Member Author

Looks good! It's exciting that it's only 1/3 the size of the original postgres image. @duanhongyi , is this ready for review and merge?

yes

@till till left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly a few questions.

Nice work! The size improvement will be significant!

Comment thread Dockerfile
Comment thread Dockerfile
Comment thread charts/database/templates/database-deployment.yaml
Comment thread rootfs/docker-entrypoint-initdb.d/003_restore_from_backup.sh
Comment thread Dockerfile
Comment thread Dockerfile

@till till left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for responding to all my questions.

Good to merge, IMO!

@duanhongyi

duanhongyi commented Nov 25, 2018

Copy link
Copy Markdown
Member Author

close, see #7

@duanhongyi duanhongyi closed this Nov 25, 2018
@Cryptophobia

Copy link
Copy Markdown
Member

@duanhongyi , I'm going to reopen merge this PR as it has better history for the approval process.

@Cryptophobia Cryptophobia reopened this Nov 25, 2018
@Cryptophobia

Copy link
Copy Markdown
Member

Merging this in now and going to build a new tag v2.7.1 so we can make Hephy Workflow v2.20 start up faster @kingdonb @duanhongyi 😁

@Cryptophobia Cryptophobia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Cryptophobia Cryptophobia merged commit 745ca31 into teamhephy:master Nov 28, 2018
@till

till commented Nov 28, 2018

Copy link
Copy Markdown

Yay!!! :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants